af78aec4f7486ecca87be1d087f04e3ce5c9d0fb,maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapUtil.java,DoapUtil,filterDevelopersOrContributorsByDoapRoles,#I18N#List#,417

Before Change


                    String role = (String) it2.next();

                    role = role.toLowerCase( Locale.ENGLISH );
                    if ( role.indexOf( getLowerCaseString( i18n, "doap.maintainer" ) ) != -1 )
                    {
                        ( (List) returnMap.get( "maintainers" ) ).add( obj );
                    }
                    else if ( role.indexOf( getLowerCaseString( i18n, "doap.developer" ) ) != -1 )
                    {
                        ( (List) returnMap.get( "developers" ) ).add( obj );
                    }
                    else if ( role.indexOf( getLowerCaseString( i18n, "doap.documenter" ) ) != -1 )
                    {
                        ( (List) returnMap.get( "documenters" ) ).add( obj );
                    }
                    else if ( role.indexOf( getLowerCaseString( i18n, "doap.translator" ) ) != -1 )
                    {
                        ( (List) returnMap.get( "translators" ) ).add( obj );
                    }
                    else if ( role.indexOf( getLowerCaseString( i18n, "doap.tester" ) ) != -1 )
                    {
                        ( (List) returnMap.get( "testers" ) ).add( obj );
                    }
                    else if ( role.indexOf( getLowerCaseString( i18n, "doap.helper" ) ) != -1 )
                    {
                        ( (List) returnMap.get( "helpers" ) ).add( obj );
                    }
                    else if ( role.indexOf( getLowerCaseString( i18n, "doap.emeritus" ) ) != -1 )
                    {
                        // Don't add as developer nor as contributor as the person is no longer involved
                    }

After Change


                for ( String role : roles )
                {
                    role = role.toLowerCase( Locale.ENGLISH );
                    if ( role.contains( getLowerCaseString( i18n, "doap.maintainer" ) ) )
                    {
                        returnMap.get( "maintainers" ).add( contributor );
                    }
                    else if ( role.contains( getLowerCaseString( i18n, "doap.developer" ) ) )
                    {
                        returnMap.get( "developers" ).add( contributor );
                    }
                    else if ( role.contains( getLowerCaseString( i18n, "doap.documenter" ) ) )
                    {
                        returnMap.get( "documenters" ).add( contributor );
                    }
                    else if ( role.contains( getLowerCaseString( i18n, "doap.translator" ) ) )
                    {
                        returnMap.get( "translators" ).add( contributor );
                    }
                    else if ( role.contains( getLowerCaseString( i18n, "doap.tester" ) ) )
                    {
                        returnMap.get( "testers" ).add( contributor );
                    }
                    else if ( role.contains( getLowerCaseString( i18n, "doap.helper" ) ) )
                    {
                        returnMap.get( "helpers" ).add( contributor );
                    }
                    else if ( role.contains( getLowerCaseString( i18n, "doap.emeritus" ) ) )
                    {
                        // Don't add as developer nor as contributor as the person is no longer involved
                    }